Skip to content

feat: add organization repository roles flag#48

Merged
goruha merged 1 commit intocloudposse-terraform-components:mainfrom
RTBHOUSE:feat/organization-repository-roles
Apr 21, 2026
Merged

feat: add organization repository roles flag#48
goruha merged 1 commit intocloudposse-terraform-components:mainfrom
RTBHOUSE:feat/organization-repository-roles

Conversation

@mtweeman
Copy link
Copy Markdown
Contributor

@mtweeman mtweeman commented Apr 21, 2026 •
edited by coderabbitai Bot
Loading

what

Adding organization repository roles flag to the variables as a follow-up of this change: cloudposse/terraform-github-repository#23.

why

Organization repository roles support was added to the terraform-github-repository module recently. Component should handle it, too.

Summary by CodeRabbit

  • New Features

    • Added a configuration option to enable organization-level repository roles; disabled by default to preserve existing behavior.

  • Chores

    • Internal alignment and formatting of module inputs for consistency — no changes to runtime behavior or exported interfaces.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 21, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f4a23298-a10e-4d79-85c4-5303e7a0dafd

📥 Commits

Reviewing files that changed from the base of the PR and between ae7d31d and a69045a.

📒 Files selected for processing (2)
  • src/main.tf
  • src/variables.tf
✅ Files skipped from review due to trivial changes (1)
  • src/variables.tf
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/main.tf
📝 Walkthrough

Walkthrough

Added a new boolean input variable organization_repository_roles_enabled (default false) and passed it into the repository module in src/main.tf; also reformatted module inputs for alignment without changing other passed values.

Changes

Cohort / File(s) Summary
Organization Repository Roles Variable
src/variables.tf		 Added organization_repository_roles_enabled variable (bool, default false) to toggle organization repository roles.
Repository Module Wiring & Formatting
src/main.tf		 Reformatted/aligned module repository inputs and added organization_repository_roles_enabled = var.organization_repository_roles_enabled to the module arguments.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nudged a tiny switch today,
A boolean hop along the way,
Wired into the module neat and small,
Quiet change—yet ready for the call. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a new Terraform boolean variable for organization repository roles functionality.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mergify mergify Bot requested review from a team April 21, 2026 09:19
@mergify mergify Bot added the triage Needs triage label Apr 21, 2026
@mergify mergify Bot removed the triage Needs triage label Apr 21, 2026
@mergify
Copy link
Copy Markdown

mergify Bot commented Apr 21, 2026

Thanks @mtweeman for creating this pull request!

A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.

While you wait, make sure to review our contributor guidelines.

Tip

Need help or want to ask for a PR review to be expedited?

Join us on Slack in the #pr-reviews channel.

@mtweeman mtweeman force-pushed the feat/organization-repository-roles branch from ae7d31d to a69045a Compare April 21, 2026 10:22
@mergify mergify Bot added the needs-test Needs testing label Apr 21, 2026
@mergify
Copy link
Copy Markdown

mergify Bot commented Apr 21, 2026

Thanks @mtweeman for creating this pull request!

A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.

While you wait, make sure to review our contributor guidelines.

Tip

Need help or want to ask for a PR review to be expedited?

Join us on Slack in the #pr-reviews channel.

@goruha goruha added this pull request to the merge queue Apr 21, 2026
Merged via the queue into cloudposse-terraform-components:main with commit 22ba453 Apr 21, 2026
14 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 21, 2026

These changes were released in v1.2.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@goruha goruha goruha approved these changes

Assignees

No one assigned

Labels

needs-test Needs testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mtweeman @goruha